Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Buildtoacr 01082024 #114

Open
wants to merge 50 commits into
base: master
Choose a base branch
from
Open

Conversation

hartou
Copy link

@hartou hartou commented Jan 9, 2024

This pull request includes a variety of changes to improve the deployment process, configuration, and infrastructure setup. The most important changes include adding parameters to the deployment script for flexibility, adding environment variables to the configuration file, and updating the infrastructure file to create a container app environment.

Deployment and configuration improvements:

  • deploy_infra.ps1: Added parameters to the deployment script for flexibility and easier customization.
  • .env.example: Added new environment variables to the configuration file for various configurations.

Infrastructure and deployment automation:

  • .github/workflows/workflow.yaml: Added a new workflow file that triggers on a push event and performs an Azure login using OIDC, useful for subsequent actions using the Azure CLI.
  • workflows/buildwithacr.yaml: Added a new workflow file that automates the build and deployment of Docker images using an Azure Container Registry.
  • infra/main.bicep: Updated the infrastructure file to add parameters, variables, and resources for creating a container app environment.

Other important changes:

  • pom.xml: Updated the version of the Jackson library to potentially incorporate new features or bug fixes.
  • .dockerignore: Added target/ directory to exclude build artifacts from Docker image.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant